-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ask for merge of last corrections for loumir/pen1_6 into ivoa-std/WD-1.6 #99
Conversation
Co-authored-by: Baptiste Cecconi <[email protected]>
trial to fix issue in generating PDF : upgraded version for upload/artifact. use v4.
…actionUploadArtifact-version-4 Update build.yml
update workflow for preview
Update in WF ?? / not clear
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, except for two minor comments that should not preclude merging.
But it seems I can't approve here (probably because I wasn't made a collaborator while I could write here as semantics chair). I'll approve if you add me.
@@ -1,4 +1,4 @@ | |||
![UCDList version](https://img.shields.io/badge/UCDList-EN--1.5-green.svg) | |||
![UCDList version](https://img.shields.io/badge/UCDList-WD--1.6-yellow.svg) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a CI skeptic and am not sure what this does, but if this is "WD" rather "PEN", there's something wrong with the underlying CI code, and that would need to be fixed.
P & \ucd{stat.correlation}& Correlation between two parameters\\ | ||
P & \ucd{stat.covariance}& Covariance between two parameters\\ | ||
P & \ucd{stat.error}& Statistical error\\ | ||
P & \ucd{stat.error.sys}& Systematic error\\ | ||
Q & \ucd{stat.falsePositive}& Rate of false alarm in detection problems\\ | ||
Q & \ucd{stat.falseNegative}& Rate of missed or false negative detection\\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a rationale for making them Q? Where would these be in a secondary position?
No description provided.